feat(generic): migration from bitnami scripts#1504
Open
leiicamundi wants to merge 51 commits intomainfrom
Open
feat(generic): migration from bitnami scripts#1504leiicamundi wants to merge 51 commits intomainfrom
leiicamundi wants to merge 51 commits intomainfrom
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
|
||
| echo "✓ Bitnami deployment ready" | ||
|
|
||
| # TODO: Generate test data for migration validation |
Contributor
There was a problem hiding this comment.
A new Todo was discovered. If it is not a priority right now, consider marking it for later attention.
TODO: # TODO: Generate test data for migration validation
🔕 React with 👎 (thumbs down) to this comment to acknowledge and prevent further reminders.
Click here to retry and refresh the list of remaining TODOs
| id: matrix | ||
| with: | ||
| ci_matrix_file: ${{ env.CI_MATRIX_FILE }} | ||
| # TODO: revert later |
Contributor
There was a problem hiding this comment.
A new Todo was discovered. If it is not a priority right now, consider marking it for later attention.
TODO: # TODO: revert later
🔕 React with 👎 (thumbs down) to this comment to acknowledge and prevent further reminders.
Click here to retry and refresh the list of remaining TODOs
|
|
||
| S3_BACKEND_BUCKET_PREFIX_EKS_SINGLE_REGION: aws/kubernetes/eks-single-region/ | ||
|
|
||
| # TODO : revert this before merge |
Contributor
There was a problem hiding this comment.
A new Todo was discovered. If it is not a priority right now, consider marking it for later attention.
TODO: # TODO : revert this before merge
🔕 React with 👎 (thumbs down) to this comment to acknowledge and prevent further reminders.
Click here to retry and refresh the list of remaining TODOs
Contributor
🔀 Skip Workflow ChecklistCheck the boxes below to skip specific workflows for this PR. Global options
Individual workflows
This checklist is automatically managed. Checked items will skip the corresponding workflow. |
leiicamundi
force-pushed
the
feature/bitnami-to-alt-migration
branch
from
6ceadcc to
e9ad358
Compare
.pre-commit-config.yaml
Outdated
| language: script | ||
| types: [terraform] | ||
| pass_filenames: false | ||
| # TODO: remove before merge - download trivy is broken |
Contributor
There was a problem hiding this comment.
A new Todo was discovered. If it is not a priority right now, consider marking it for later attention.
TODO: # TODO: remove before merge - download trivy is broken
🔕 React with 👎 (thumbs down) to this comment to acknowledge and prevent further reminders.
Click here to retry and refresh the list of remaining TODOs
- Add --connect-timeout 5 --max-time 10 to all readiness/wait curl calls - Add --connect-timeout 5 --max-time 30 to all action curl calls (deploy, create) - Fix 'totalItems' grep pattern (API returns 'totalItems' not 'total') - Remove -f flag from HTTP_CODE curls to avoid 409+000 concatenation - Add ES indexing retry loop in verify job (up to 12 retries x 10s) - Reduce WebModeler readiness retries from 6x10s to 3x5s Seed job: 8m58s -> 43s Verify job: timeout -> 39s
The image camunda/keycloak:quay-optimized-26.5.0 does not exist. Reverts the erroneous bump from commit 0f6ca76.
leiicamundi
force-pushed
the
feature/bitnami-to-alt-migration
branch
2 times, most recently
from
d5c857a to
8a1ab51
Compare
The _reindex API fails for some empty indices due to mapping-only incompatibilities (e.g. optimize-process-instance-migration-seed-process_v8). This caused the entire es-restore job to fail with backoffLimit reached. Changes: - Check source doc count before reindexing; skip indices with 0 docs - Remove -f flag from curl to show error responses for diagnostics - Parse response body for errors instead of relying on curl exit code - Report skipped empty indices in final summary
leiicamundi
force-pushed
the
feature/bitnami-to-alt-migration
branch
from
8a1ab51 to
ba28560
Compare
When CAMUNDA_HELM_CHART_VERSION is the default placeholder (0.0.0-snapshot-alpha), helm upgrade fails because that version doesn't exist in the public Helm repo. Fix: auto-detect the installed chart version from helm list and use it for --version in both helm_upgrade and helm_rollback_from_backup.
…clusion When using --reuse-values, the existing identityPostgresql.enabled=true and identityKeycloak.postgresql.enabled=true are carried over from the Bitnami install. These conflict with external database / operator configs. Explicitly set them to false in the operator-based values overlays.
Signed-off-by: Leo <153937047+leiicamundi@users.noreply.github.com>
… client secrets - ES restore: Add 'camunda-' to index listing, cleanup, and alias grep patterns. Camunda 8.8 introduced a new 'camunda-*' index prefix for indices like camunda-user, camunda-authorization, camunda-tenant, etc. These were missed by the old grep pattern, causing Zeebe to fail with index_not_found_exception on *_alias indices. - Identity: Generate explicit Helm overrides for client secrets (admin, console, connectors, orchestration). When identityKeycloak.enabled switches to false, the chart stops auto-rendering client secret env vars, causing Identity to crash with 'Please supply a valid client secret for Connectors'.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.